Skip to content

Fic loc paths from projects for localization and remove res swix#63

Merged
spebl merged 10 commits intodev15from
dev/spebl/swixloc
Sep 29, 2017
Merged

Fic loc paths from projects for localization and remove res swix#63
spebl merged 10 commits intodev15from
dev/spebl/swixloc

Conversation

@spebl
Copy link
Copy Markdown

@spebl spebl commented Sep 28, 2017

Also added option to specify if it's a product component for vs installer and added mission dlls to setversion.ps1

@spebl spebl requested review from a team and LukaszMendakiewicz September 28, 2017 23:46
@spebl
Copy link
Copy Markdown
Author

spebl commented Sep 28, 2017

Note: The 4 strings in GoogleTestAdapter.Common.Dynamic.dll aren't currently being localized and will still always appear in english. Working on fixing that now but wanted to get the main chunk in. This has to do with the way we load the DLL in Common.

<BypassVsixValidation>true</BypassVsixValidation>
<DeployExtension Condition="'$(TestAdapterFlavor)' != 'TAfGT'">false</DeployExtension>
<IsProductComponent>true</IsProductComponent>
<IsProductComponent>$(IsProductComponent)</IsProductComponent>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be more canonical to do Condition and set true/false accordingly?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is interesting since it's using the replacement from Microbuild for this actually. I do agree we should set it default here in case this isn't set (for local build for instance).

@@ -1,42 +0,0 @@
<?xml version="1.0" encoding="utf-8"?>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need these files any more? Because we now set IsProductComponent?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They were never used actually. I had them there since I was basing the loc after a different example which ended up being unnecessary.

@LukaszMendakiewicz LukaszMendakiewicz added this to the v0.10.1.1-tafgt milestone Sep 29, 2017
@spebl spebl merged commit eb8121b into dev15 Sep 29, 2017
@bobbrow bobbrow deleted the dev/spebl/swixloc branch November 25, 2025 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants